Skip to content

docs: document pcov and imagick incompatibilities in known issues#2492

Merged
alexandre-daubois merged 1 commit into
php:mainfrom
alexandre-daubois:docs/pcov-imagick-known-issues
Jul 10, 2026
Merged

docs: document pcov and imagick incompatibilities in known issues#2492
alexandre-daubois merged 1 commit into
php:mainfrom
alexandre-daubois:docs/pcov-imagick-known-issues

Conversation

@alexandre-daubois

Copy link
Copy Markdown
Member

pcov is not thread-safe: it swaps the process-global zend_compile_file and zend_execute_ex engine hooks without synchronization, which races across threads and crashes (SIGSEGV) in classic mode (krakjoe/pcov#132). Imagick is unstable because ImageMagick's OpenMP threads conflict with FrankenPHP's threading model.

Copilot AI review requested due to automatic review settings June 26, 2026 09:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Documents two additional extension-related incompatibilities/instability cases to help FrankenPHP users avoid known crash scenarios in threaded/classic mode and choose safer workarounds.

Changes:

  • Adds pcov to the “Unsupported PHP extensions” list with a link to the known SIGSEGV/thread-safety issue and a CLI-only coverage workaround.
  • Adds imagick to the “Buggy PHP extensions” list with mitigations (limit threads / disable OpenMP).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/known-issues.md Outdated
Comment thread docs/known-issues.md
Comment thread docs/known-issues.md
Copilot AI review requested due to automatic review settings July 10, 2026 07:09
@alexandre-daubois alexandre-daubois force-pushed the docs/pcov-imagick-known-issues branch from e7f40c3 to 92e0ebe Compare July 10, 2026 07:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread docs/known-issues.md Outdated
Copilot AI review requested due to automatic review settings July 10, 2026 07:13
@alexandre-daubois alexandre-daubois force-pushed the docs/pcov-imagick-known-issues branch from 92e0ebe to 970e982 Compare July 10, 2026 07:13
pcov is not thread-safe: it swaps the process-global zend_compile_file and zend_execute_ex engine hooks without synchronization, which races across threads and crashes (SIGSEGV) in classic mode (krakjoe/pcov#132). Imagick is unstable because ImageMagick's OpenMP threads conflict with FrankenPHP's threading model.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@alexandre-daubois alexandre-daubois merged commit bc9c6db into php:main Jul 10, 2026
2 checks passed
@alexandre-daubois alexandre-daubois deleted the docs/pcov-imagick-known-issues branch July 10, 2026 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants